-
Notifications
You must be signed in to change notification settings - Fork 77
Fix CI #640
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix CI #640
Conversation
|
@copilot Try to fix the CPP build error |
|
@Happypig375 I've opened a new pull request, #641, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
@copilot It seems that AngouriMath.CPP.Exporting isn't generating native binaries to be copied over to AngouriMath.CPP.Importing in the CI. What happened? |
|
@Happypig375 I've opened a new pull request, #642, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
@WhiteBlackGoose Why are some of these projects not in the solution? |
|
The C++ projects aren't really worth maintaining IMO, I don't think anybody uses those unironically. If some .net projects aren't in the solution, it's probably unintentional. At some point I stopped using VS so I didn't need the solution file any more. Feel free to add them |
|
Also I recommend using squash and merge so you get one commit per PR instead of keeping all temporary commits in main |
|
I have no idea how including more projects in the solution fixes C++ Test CI but sure I guess |
|
@WhiteBlackGoose push another preview to nuget please |
No description provided.